Skip to content

Update to latest commit of argo-rollouts-manager '67002e569d7f7de26a60a1f5a8d0b4924931b4da' - #1269

Merged
openshift-merge-bot[bot] merged 2 commits into
redhat-developer:masterfrom
jgwest:upgrade-rollouts-manager
Sep 1, 2026
Merged

Update to latest commit of argo-rollouts-manager '67002e569d7f7de26a60a1f5a8d0b4924931b4da'#1269
openshift-merge-bot[bot] merged 2 commits into
redhat-developer:masterfrom
jgwest:upgrade-rollouts-manager

Conversation

@jgwest

@jgwest jgwest commented Aug 31, 2026

Copy link
Copy Markdown
Member

Update to most recent 'argo-rollouts-manager' commit: argoproj-labs/argo-rollouts-manager@67002e5

@jgwest
Update to latest commit of argo-rollouts-manager '67002e569d7f7de26a6…
2b95bb3 
…0a1f5a8d0b4924931b4da'

Signed-off-by: Jonathan West <jgwest@gmail.com>
@openshift-ci
openshift-ci Bot requested review from anandrkskd and svghadi August 31, 2026 22:08
@coderabbitai

coderabbitai Bot commented Aug 31, 2026 •
edited
Loading

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 382c0d25-7ca8-4ae6-ae68-daa09eba8c23

📥 Commits

Reviewing files that changed from the base of the PR and between 2b95bb3 and 2147c80.

📒 Files selected for processing (1)
  • scripts/run-rollouts-e2e-tests.sh
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • argoproj-labs/argocd-operator (manual)
🚧 Files skipped from review as they are similar to previous changes (1)
  • scripts/run-rollouts-e2e-tests.sh

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated release metadata to reflect the latest publication timestamp.
    • Updated rollout components used by the project to incorporate the latest available improvements.
    • Aligned rollout tooling versions across release and validation workflows.
    • Improved consistency between packaged release information and automated rollout checks, supporting more reliable deployment validation.

Walkthrough

The changes update the Argo Rollouts Manager dependency and E2E test pins. The CSV creation timestamp is also updated.

Changes

Release alignment

Layer / File(s) Summary
Version and metadata updates
go.mod, scripts/run-rollouts-e2e-tests.sh, bundle/manifests/gitops-operator.clusterserviceversion.yaml		 The Argo Rollouts Manager dependency and E2E pins reference newer revisions. The CSV createdAt timestamp is updated to 2026-08-31T22:08:00Z.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 2147c

This change updates the argo-rollouts-manager dependency and its end-to-end test script without supplied evidence of a concrete production or correctness risk; no actionable merge-blocking risk remains.

Suggested reviewers: anandrkskd, svghadi

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the update to the latest argo-rollouts-manager commit and matches the stated objective.
Description check ✅ Passed The description directly explains the argo-rollouts-manager dependency update and includes the target commit link.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files.

Comment @coderabbitai help to get the list of available commands.

@jgwest
chore: update rollouts plugin commit
2147c80 
Signed-off-by: Jonathan West <jgwest@gmail.com>
@jgwest

jgwest commented Sep 1, 2026

Copy link
Copy Markdown
Member Author 

�[36mINFO�[0m[2026-08-31T22:20:53Z] Acquiring leases for test kuttl-sequential: [gitops-aws-quota-slice] 
�[31mERRO�[0m[2026-09-01T00:20:53Z] error: Failed to acquire resource, current capacity: 0 free, 10 leased

/retest

@olivergondza

olivergondza commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

/lgtm
/approve

@openshift-ci openshift-ci Bot added the lgtm label Sep 1, 2026
@openshift-ci

openshift-ci Bot commented Sep 1, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: olivergondza

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved label Sep 1, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit 2aeada6 into redhat-developer:master Sep 1, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anandrkskd anandrkskd Awaiting requested review from anandrkskd
@svghadi svghadi Awaiting requested review from svghadi

Assignees

@olivergondza olivergondza

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jgwest @olivergondza